チュートリアル、起動時のイベントログ関連画面の追加 #1063

ghost · 2022-07-01T07:47:25Z

Issue 番号 / Issue ID

Close #

目的 / Purpose

チュートリアル、起動時のイベントログ関連画面の追加

Screen1	Screen2

変更内容 / Changes

イベントログ収集の協力依頼の画面追加
イベントログ設定を追加
- このアプリについて画面（設定画面）からの遷移は別途対応
- A11y対応は最小限対応。同じく別途対応する。
初回起動時、アプリアップデートしてからの起動時の流れを実装

破壊的変更をもたらしますか / Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request の種類 / Pull Request type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

確認事項 / What to check

[ ]

その他 / Other information

Internal IDs:

Product Backlog Item 7403

ghost · 2022-07-01T07:53:01Z

Covid19Radar/Covid19Radar.Android/MainActivity.cs


- AppCompatDelegate.DefaultNightMode = AppCompatDelegate.ModeNightNo;


MainApplecationのOnCreate()に移動しています
この位置だとMainActivityのOnCreate()が2回呼ばれる。
base.OnCreate()の前にしたらこれは解消するが、初回起動時に反映されない
ので、MainApplecationのOnCreate()で行うことで解消した

https://github.com/cocoa-mhlw/cocoa/pull/1063/files#diff-53211e854c73f0587cc49490a1b3825d420966627d7f6501e12c11281f89460dR76

cocoa-dev003 · 2022-07-01T09:44:24Z

Covid19Radar/Covid19Radar/ViewModels/Settings/EventLogSettingPageViewModel.cs

+ {
+ _sendEventLogStateRepository.SetSendEventLogState(EventType.ExposureNotified, ExposureNotifyIsChecked ? SendEventLogState.Enable : SendEventLogState.Disable);
+
+ foreach (var eventType in EventType.All)


仕様の理解足りてないかもですが、この処理ってなんのためですか？

現状つかっている設定値がEventType.ExposureNotifiedだけですが、
EventType.Allに仮の値が存在しているため、未使用のものはDisableにしています。

複数設定値を考慮しておきたいとのことなのでこのままにしていましたが、使わない値は消しておいたほうが良いですね。
不要な設定値は削除するようにします。

~~修正しました~~
~~9084cf1~~

この処理も不要ですね。

再度修正しました

9084cf1
c43e212

cocoa-dev003 · 2022-07-01T09:46:02Z

Covid19Radar/Covid19Radar/Resources/AppResources.zh-Hans.resx

+ <comment>・送信は匿名で行われます。</comment>
+ </data>
+ <data name="EventLogCooperationPageBullets2" xml:space="preserve">
+ <value>- 发送内容中不包括可以识别您和与您接触的阳性登记人的信息。</value>


中国語って今まで日本語と同じように黒丸にしていませんでしたっけ。
この画面だけ違うの違和感あるので、どちらかに揃えたほうがいいかと思います。

修正しました
c9afc73

cocoa-dev003 · 2022-07-01T09:51:30Z

Covid19Radar/Covid19Radar/ViewModels/Tutorial/EventLogCooperationPageViewModel.cs

+
+ try
+ {
+ foreach (var eventType in EventType.All)


「あとで設定する」を選択したらNotSetのままだと思ってたんですが、私の仕様の理解違いましたかね？
ACに「アプリ起動時に、on/offどちらにも設定していない送信項目があった場合に都度表示すること」と記載あるので、ちょっとしつこいですがあとで設定するを選択した人には起動時に何回も表示するんだと思ってました。。

後で設定する＝設定画面であとで設定する
になるものと思っていました。
確認、調整します。

あとで設定するを選択した人には起動時に何回も表示する

このような処理に修正しました。
27620d3
d07cd11

Covid19Radar/Covid19Radar/Views/Tutorial/TutorialPage4.xaml

...ar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/EventLogCooperationPageViewModelTests.cs

sonarcloud · 2022-07-05T06:38:18Z

SonarCloud Quality Gate failed.

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

78.9% Coverage
0.0% Duplication

cocoa-dev004 added 3 commits June 28, 2022 16:41

Fixed OnCreate() being called twice

1ecbc12

Fixed page transition of splash page

c5411ad

Add event log page

83aeec8

ghost requested review from a user, cocoa-dev003 and cocoa-dev009 July 1, 2022 07:47

ghost self-assigned this Jul 1, 2022

ghost commented Jul 1, 2022

View reviewed changes

cocoa-dev003 reviewed Jul 1, 2022

View reviewed changes

cocoa-dev004 added 5 commits July 4, 2022 08:44

Fixed bullets symbol of zh-hans

c9afc73

Remove unuse EventType

9084cf1

Fixed event log save

c43e212

Fixed set-later command

27620d3

Fixed splash page transition when setup later

d07cd11

cocoa-dev003 approved these changes Jul 5, 2022

View reviewed changes

ghost reviewed Jul 5, 2022

View reviewed changes

Covid19Radar/Covid19Radar/Views/Tutorial/TutorialPage4.xaml Outdated Show resolved Hide resolved

ghost reviewed Jul 5, 2022

View reviewed changes

...ar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/EventLogCooperationPageViewModelTests.cs Outdated Show resolved Hide resolved

fixed typo

5e884df

ghost approved these changes Jul 5, 2022

View reviewed changes

ghost merged commit 53fe646 into develop Jul 5, 2022

ghost deleted the feature/event-log-pages branch July 5, 2022 06:28

This pull request was closed.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

チュートリアル、起動時のイベントログ関連画面の追加 #1063

チュートリアル、起動時のイベントログ関連画面の追加 #1063

ghost commented Jul 1, 2022 •

edited by ghost

Loading

ghost Jul 1, 2022 •

edited by ghost

Loading

cocoa-dev003 Jul 1, 2022

ghost Jul 3, 2022

ghost Jul 4, 2022 •

edited by ghost

Loading

ghost Jul 4, 2022

cocoa-dev003 Jul 1, 2022

ghost Jul 4, 2022

cocoa-dev003 Jul 1, 2022

ghost Jul 3, 2022

ghost Jul 4, 2022 •

edited by ghost

Loading

sonarcloud bot commented Jul 5, 2022


		AppCompatDelegate.DefaultNightMode = AppCompatDelegate.ModeNightNo;

チュートリアル、起動時のイベントログ関連画面の追加 #1063

チュートリアル、起動時のイベントログ関連画面の追加 #1063

Conversation

ghost commented Jul 1, 2022 • edited by ghost Loading

Issue 番号 / Issue ID

目的 / Purpose

変更内容 / Changes

破壊的変更をもたらしますか / Does this introduce a breaking change?

Pull Request の種類 / Pull Request type

確認事項 / What to check

その他 / Other information

ghost Jul 1, 2022 • edited by ghost Loading

Choose a reason for hiding this comment

cocoa-dev003 Jul 1, 2022

Choose a reason for hiding this comment

ghost Jul 3, 2022

Choose a reason for hiding this comment

ghost Jul 4, 2022 • edited by ghost Loading

Choose a reason for hiding this comment

ghost Jul 4, 2022

Choose a reason for hiding this comment

cocoa-dev003 Jul 1, 2022

Choose a reason for hiding this comment

ghost Jul 4, 2022

Choose a reason for hiding this comment

cocoa-dev003 Jul 1, 2022

Choose a reason for hiding this comment

ghost Jul 3, 2022

Choose a reason for hiding this comment

ghost Jul 4, 2022 • edited by ghost Loading

Choose a reason for hiding this comment

sonarcloud bot commented Jul 5, 2022

ghost commented Jul 1, 2022 •

edited by ghost

Loading

ghost Jul 1, 2022 •

edited by ghost

Loading

ghost Jul 4, 2022 •

edited by ghost

Loading

ghost Jul 4, 2022 •

edited by ghost

Loading